fix(marketing): keep Grok mark clear of mobile hero copy - #4542
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Approved Pure CSS positioning changes adjusting where a decorative element appears on mobile views of the marketing page. No logic or runtime behavior changes - just visual layout adjustments. You can customize Macroscope's approvability policy. Learn more. |
Summary
Fixes #4234
Before and after
Mobile (393 × 852)
Desktop (rendered at 1440 × 1000)
The screenshots are hosted on the
pr-assets-4234branch in the fork so review evidence stays out of the product diff.Verification
pnpm --filter @t3tools/marketing typecheckpnpm --filter @t3tools/marketing buildgit diff --checkNote
Low Risk
CSS-only marketing hero layout at two breakpoints; no logic, data, or desktop behavior changes.
Overview
Repositions the Grok floating harness icon on narrow viewports so it no longer sits over the mobile hero headline.
At ≤820px,
.hf-grokmoves from the left stack beside the title to the top row, centered above the headline (top: 44px,left: calc(50% - 39px)). The layout comment is updated to describe three marks above the headline and two by the CTA.At ≤340px, Grok’s compact placement is adjusted (
top: 57px,left: calc(50% - 26px)) so it stays centered with the smaller mark size. Desktop and other float marks are unchanged.Reviewed by Cursor Bugbot for commit 858b935. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Reposition Grok floating mark to top-center on mobile hero
On mobile, the
.hf-grokfloating mark was overlapping the hero copy. It is now positioned near the top of the hero and centered horizontally usingcalc(50% - 39px), moving it away from the headline and CTA. For viewports ≤340px, a separate rule centers it atcalc(50% - 26px)with a slightly lower top offset.Macroscope summarized 858b935.